create by DevLabStudy · Pull Request #17 · DevLabStudy/orbit

DevLabStudy · 2026-02-02T16:12:21Z

No description provided.

Add screenshots

Update README formatting and feature list

Added a new section for Login with an image.

Add Login section to README

Updated the login image width in the README.

Increase login image width to 800px

Update README.md

Fix formatting of screenshots section in README

Update README.md

Create docker-publish.yml

Update docker-publish.yml

Add Patreon funding option alongside Buy Me a Coffee

Delete orbit.db

Add stacks, logs, space background

Add stacks, container logs, space background

Update .gitignore to include additional patterns

Copilot

Pull request overview

Adds Stack (Docker Compose) management and container log viewing to the Orbit Control dashboard, alongside significant UI styling updates and repository housekeeping.

Changes:

Added Stacks tab with create/start/stop/restart/remove actions and backend endpoints to manage Compose projects.
Added container log viewing (UI modal + /container/<id>/logs endpoint).
Updated UI styling (space background, banners, modal extensions) and added a Docker publish GitHub Actions workflow.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 13 comments.

Show a summary per file

File	Description
`templates/index.html`	Adds new UI elements (space background, GitHub banner) and new modals for stack creation and logs display.
`static/js/dashboard.js`	Implements Stacks UI logic, stack actions, and container logs modal/refresh behavior.
`app.py`	Adds stack discovery and stack lifecycle endpoints, plus a container logs endpoint; bumps app version.
`README.md`	Updates formatting and screenshots section, but now diverges from the implemented features and has HTML/link issues.
`.gitignore`	Adds DB ignore rules, but the DB is still present in the PR and should be removed from tracking.
`.github/workflows/docker-publish.yml`	Adds image build/push workflow.
`.github/FUNDING.yml`	Adds Patreon funding option.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-02T16:19:12Z

app.py

+        stack_dir = os.path.join(STACKS_PATH, name)
+        os.makedirs(stack_dir, exist_ok=True)
+        compose_file = os.path.join(stack_dir, 'docker-compose.yml')
+        with open(compose_file, 'w') as f:
+            f.write(compose_content)


stack_create() uses the user-provided name directly in os.path.join(STACKS_PATH, name) and then writes docker-compose.yml. This allows path traversal (e.g., ../...) or names containing path separators, which can write outside /app/stacks. Validate/sanitize name (e.g., strict regex like [a-z0-9][a-z0-9_-]*), and enforce that the resolved path stays under STACKS_PATH before creating directories/files.

Copilot · 2026-02-02T16:19:12Z

app.py

+        result = subprocess.run(['docker', 'compose', '-p', name, '-f', compose_file, 'up', '-d'],
+                                capture_output=True, text=True)
+        if result.returncode != 0:


stack_create() shells out to docker compose ... up -d via subprocess.run(...) without a timeout and without handling FileNotFoundError when the docker CLI / compose plugin is missing. Given the current runtime image installs only Python deps, this is likely to fail or hang indefinitely. Consider either implementing stack creation via the Docker SDK, or ensure the image includes the Docker CLI + compose plugin and add timeout=... + explicit error handling so the API returns a clear message.

Copilot · 2026-02-02T16:19:13Z

static/js/dashboard.js

        const res = await fetch('https://api.github.com/repos/TobiMessi/orbit/releases/latest');
        if (!res.ok) return;
-
        const data = await res.json();
        const latestVersion = data.tag_name.replace('v', '');
-
-        if (latestVersion !== currentVersion) {
+        if (latestVersion !== "1.1.0") {


checkForUpdate() compares GitHub’s latest tag against a hard-coded version string. This will drift on every release and duplicates VERSION from the backend. Fetch the current version from /api/version and compare against that value instead of embedding it in JS.

Copilot · 2026-02-02T16:19:13Z

.github/workflows/docker-publish.yml

+on:
+  push:
+    branches: [ "main", "add-screenshots" ] # Reaguje na Twoje obecne branche
+


This workflow publishes on pushes to both main and add-screenshots. Publishing images from a non-release branch increases the chance of pushing unreviewed or experimental builds to :latest. Consider limiting to main and/or tags/releases.

Copilot · 2026-02-02T16:19:13Z

.github/workflows/docker-publish.yml

+
+jobs:
+  build:
+    runs-on: [self-hosted, hp-build]


The job runs on a self-hosted runner while using Docker Hub secrets. Self-hosted runners have a larger trust surface than GitHub-hosted runners. If self-hosted is required, consider adding environment protections/approvals and restricting runner access to reduce secret-exposure risk.

Suggested change

runs-on: [self-hosted, hp-build]

runs-on: ubuntu-latest

Copilot · 2026-02-02T16:19:14Z

README.md

 </details>



The screenshots section has an extra closing </details> tag (there’s one closing tag at line 104 and another at line 107). This breaks the HTML structure in the rendered README; remove the redundant closing tag.

Suggested change

</details>

Copilot · 2026-02-02T16:19:15Z

README.md

 **Monitor and manage your Docker containers, images, volumes, and networks from a beautiful web interface.**

 [Features](#-features) • [Installation](#-installation) • [Usage](#-usage) • [Roadmap](#-roadmap)

 </div>


The top-of-README quick links use anchors like #-features / #-installation, but the headings now render to anchors like #features / #installation on GitHub. Update these link targets (or restore previous heading formatting) so the links work.

Copilot · 2026-02-02T16:19:15Z

templates/index.html

+            0%, 100% { opacity: 1; }
+            50% { opacity: 0.5; }
+        }
+


The new background animations (e.g., twinkle and other infinite animations in this file) don’t respect reduced-motion preferences. Add a @media (prefers-reduced-motion: reduce) override to disable or significantly reduce these animations for accessibility.

Suggested change

@media (prefers-reduced-motion: reduce) {

.stars,

.stars2,

.planet1,

.planet2 {

animation: none !important;

}

}

Copilot · 2026-02-02T16:19:15Z

app.py

+                           capture_output=True, text=True)
+            os.remove(compose_file)
+            try: os.rmdir(stack_dir)
+            except: pass


Except block directly handles BaseException.

Suggested change

except: pass

except OSError: pass

Copilot · 2026-02-02T16:19:16Z

app.py

+                           capture_output=True, text=True)
+            os.remove(compose_file)
+            try: os.rmdir(stack_dir)
+            except: pass


'except' clause does nothing but pass and there is no explanatory comment.

Suggested change

except: pass

except OSError: pass

DevLabStudy added 25 commits January 24, 2026 10:54

Merge pull request #3 from TobiMessi/add-screenshots

2e880bb

Add screenshots

Update README formatting and feature list

ed3a19a

Merge pull request #4 from TobiMessi/TobiMessi-patch-1

0b83db5

Update README formatting and feature list

Add Login section to README

665b2d6

Added a new section for Login with an image.

Merge pull request #5 from TobiMessi/TobiMessi-patch-2

3692cbf

Add Login section to README

Increase login image width to 800px

3046077

Updated the login image width in the README.

Merge pull request #6 from TobiMessi/TobiMessi-patch-1

7daf19b

Increase login image width to 800px

Update README.md

c3f3d29

Merge pull request #7 from TobiMessi/TobiMessi-patch-2

15f274a

Update README.md

Fix formatting of screenshots section in README

88862d0

Merge pull request #8 from TobiMessi/TobiMessi-patch-3

1826c6c

Fix formatting of screenshots section in README

Update README.md

aa00190

Merge pull request #9 from TobiMessi/TobiMessi-patch-4

4348572

Update README.md

Create docker-publish.yml

27dc7b4

Merge pull request #10 from TobiMessi/TobiMessi-patch-1

4d914da

Create docker-publish.yml

Update docker-publish.yml

758dde3

Merge pull request #11 from TobiMessi/TobiMessi-patch-1

c51adcc

Update docker-publish.yml

Add Patreon funding option alongside Buy Me a Coffee

dd59f7b

Merge pull request #12 from TobiMessi/TobiMessi-patch-1

8bc63a0

Add Patreon funding option alongside Buy Me a Coffee

Delete orbit.db

e5c96ef

Merge pull request #13 from TobiMessi/TobiMessi-patch-2

52d3ead

Delete orbit.db

Add stacks, logs, space background

805e2ac

Merge pull request #15 from TobiMessi/feature-stacks-v2

0eb5640

Add stacks, logs, space background

Add stacks, container logs, space background

08793ce

Merge pull request #16 from TobiMessi/feature-space-stacks

d1cb847

Add stacks, container logs, space background

Copilot AI review requested due to automatic review settings February 2, 2026 16:12

Copilot started reviewing on behalf of DevLabStudy February 2, 2026 16:12 View session

DevLabStudy added 2 commits February 2, 2026 17:14

Update .gitignore to include additional patterns

ce76d56

Merge pull request #18 from TobiMessi/TobiMessi-patch-1

30c29aa

Update .gitignore to include additional patterns

DevLabStudy closed this Feb 2, 2026

Copilot AI reviewed Feb 2, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

create#17

create#17
DevLabStudy wants to merge 27 commits intofeature-stacks-logsfrom
main

DevLabStudy commented Feb 2, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Copilot AI Feb 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

+        @media (prefers-reduced-motion: reduce) {
+            .stars,
+            .stars2,
+            .planet1,
+            .planet2 {
+                animation: none !important;
+            }
+        }

Uh oh!

Conversation

DevLabStudy commented Feb 2, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant